Skip to content

Comments

feat: atproto oauth#273

Merged
danielroe merged 44 commits intonpmx-dev:mainfrom
zeucapua:feat/atproto-oauth
Jan 31, 2026
Merged

feat: atproto oauth#273
danielroe merged 44 commits intonpmx-dev:mainfrom
zeucapua:feat/atproto-oauth

Conversation

@zeucapua
Copy link
Contributor

@zeucapua zeucapua commented Jan 29, 2026 •
edited
Loading

Implements AT Protocol OAuth using @atproto/api and @atproto/oauth-client-node. The form allows users to login with a handle from the app, create an account through selfhosted.social, or go through Bluesky.

Closes #77

npmx-oauth.mp4

Notes:

  • requires NUXT_SESSION_PASSWORD env variable to encrypt cookies
  • login will fail on preview due to the redirect URIs not matching the generated preview URL

@vercel
Copy link

vercel bot commented Jan 29, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 31, 2026 8:34am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Jan 31, 2026 8:34am
npmx-lunaria Ignored Ignored Preview Jan 31, 2026 8:34am

Request Review

43081j
43081j reviewed Jan 29, 2026
View reviewed changes
43081j
43081j reviewed Jan 29, 2026
View reviewed changes
43081j
43081j reviewed Jan 29, 2026
View reviewed changes
43081j
43081j reviewed Jan 29, 2026
View reviewed changes
@43081j
Copy link
Contributor

43081j commented Jan 29, 2026

just a couple of minor comments but looks good to me, pretty straightforward! nice work 🎉

jonathanyeong
jonathanyeong reviewed Jan 29, 2026
View reviewed changes
fatfingers23
fatfingers23 reviewed Jan 29, 2026
View reviewed changes
@fatfingers23
Copy link
Contributor

fatfingers23 commented Jan 31, 2026

I think we are good to get this merged when someone has a chance to take a final look! 🥳

@github-actions
Copy link

github-actions bot commented Jan 31, 2026 •
edited
Loading

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
lunaria/files/en-US.json Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@danielroe danielroe enabled auto-merge January 31, 2026 08:46
@danielroe danielroe disabled auto-merge January 31, 2026 08:47
@danielroe danielroe merged commit 4e9cd16 into npmx-dev:main Jan 31, 2026
12 of 13 checks passed
@pushcx pushcx mentioned this pull request Jan 31, 2026
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fatfingers23 fatfingers23 fatfingers23 approved these changes

@danielroe danielroe danielroe approved these changes

@43081j 43081j Awaiting requested review from 43081j

+2 more reviewers

@Kai-ros Kai-ros Kai-ros requested changes

@jonathanyeong jonathanyeong jonathanyeong approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: atproto oauth

6 participants

@zeucapua @43081j @danielroe @fatfingers23 @jonathanyeong @Kai-ros